tests/installed: Fix TESTS= being empty
authorColin Walters <walters@verbum.org>
Mon, 26 Mar 2018 17:23:25 +0000 (13:23 -0400)
committerAtomic Bot <atomic-devel@projectatomic.io>
Mon, 26 Mar 2018 19:02:54 +0000 (19:02 +0000)
I broke this in https://github.com/ostreedev/ostree/pull/1509/commits/9b55aaea6f34b7094c44932a3c2e1cf2d54634fd
I'd obviously tested *setting* it locally worked, but I didn't test that
not having it set ran all the tests.

I don't understand why we were doing the `+ ` pattern before; let's
just check if it's empty.

Closes: #1516
Approved by: jlebon

tests/installed/run.sh

index 38a9d769869359345a35472aa391dd37d4c2d14e..8c7fe4d85e09ab0816fbd8d4155e014479384590 100755 (executable)
@@ -4,7 +4,7 @@ set -xeuo pipefail
 
 dn=$(dirname $0)
 for tn in ${dn}/itest-*.sh; do
-    if [ -n "${TESTS}" ]; then
+    if [ -n "${TESTS:-}" ]; then
       tbn=$(basename "$tn" .sh)
       tbn=" ${tbn#itest-} "
       if [[ " $TESTS " != *$tbn* ]]; then